-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use require
wherever possible
#19
base: master
Are you sure you want to change the base?
Conversation
This is the only API that needs conversion to a module.
@danports what is |
Also moved the two modules's source code files to .lua.
Also removed some blank lines that had whitespace.
|
Also trimmed some whitespace and sent unused iterators to nil.
This is out of scope for this PR, but maybe we should figure out a better solution for configuration settings. The default |
Trimmed whitespace, split functions into a separate module.
975b52c
to
cbccefc
Compare
I'm going to split |
Refactored to use `require`, as well as split into a separate module. Some whitespace was trimmed.
Also made it use `require` and trimmed whitespace.
There's only two modules that needs conversion, and the rest should be simple. We'll also need to move the
startup
scripts tostartup.d/rail*.lua
and add a dependency onstartupmgr
.See also: danports/prism-core#27